Skip to content

Fix error serializing empty reference from Archetypes content. [7.x.x]#1915

Merged
mauritsvanrees merged 1 commit into7.x.xfrom
maurits-issue-1913-at-ref-none
May 13, 2025
Merged

Fix error serializing empty reference from Archetypes content. [7.x.x]#1915
mauritsvanrees merged 1 commit into7.x.xfrom
maurits-issue-1913-at-ref-none

Conversation

@mauritsvanrees
Copy link
Member

Fixes #1913

@mister-roboto
Copy link

@mauritsvanrees thanks for creating this Pull Request and helping to improve Plone!

TL;DR: Finish pushing changes, pass all other checks, then paste a comment:

@jenkins-plone-org please run jobs

To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass, but it takes 30-60 min. Other CI checks are usually much faster and the Plone Jenkins resources are limited, so when done pushing changes and all other checks pass either start all Jenkins PR jobs yourself, or simply add the comment above in this PR to start all the jobs automatically.

Happy hacking!

@mauritsvanrees
Copy link
Member Author

@davisagli The tests fail due to using no longer available action versions and Ubuntu 20.
Can we ignore that for this small fix?

I would rather make an internal release than put any time in fixing the CI for this old version.

Copy link
Member

@davisagli davisagli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mauritsvanrees Looks okay to me.

@mauritsvanrees mauritsvanrees merged commit f9fe543 into 7.x.x May 13, 2025
2 of 13 checks passed
@mauritsvanrees mauritsvanrees deleted the maurits-issue-1913-at-ref-none branch May 13, 2025 10:23
@mauritsvanrees
Copy link
Member Author

Thanks for the review.
I have released plone.restapi 7.9.1 with this fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants